Skip to content

feat: proper exception on invalid soc statement#406

Merged
hfudev merged 1 commit intoespressif:mainfrom
horw:feat/raise-error-on-wrong-soc-statement
Mar 2, 2026
Merged

feat: proper exception on invalid soc statement#406
hfudev merged 1 commit intoespressif:mainfrom
horw:feat/raise-error-on-wrong-soc-statement

Conversation

@horw
Copy link
Copy Markdown
Collaborator

@horw horw commented Mar 2, 2026

Description

This fix will help make the error more explicit, since it’s possible that the user inputs a SOC statement, which doesn’t work.

https://github.com/espressif/idf-extra-components/actions/runs/22336912154/job/64632505958?pr=681

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@horw horw requested a review from hfudev March 2, 2026 05:53
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 2, 2026

Title Coverage Tests Skipped Failures Errors Time
Qemu Coverage 7 0 💤 0 ❌ 0 🔥 25.43s ⏱️
3.14 X64 Coverage 103 14 💤 2 ❌ 0 🔥 13m 46s ⏱️
3.10 ARM64 Coverage 103 16 💤 3 ❌ 0 🔥 10m 45s ⏱️

@hfudev hfudev merged commit 60e9456 into espressif:main Mar 2, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants